Skip to content

ARROW-6209: [Java] Extract set null method to the base class for fixed width vectors#5064

Closed
liyafan82 wants to merge 1 commit intoapache:masterfrom
liyafan82:fly_0812_null
Closed

ARROW-6209: [Java] Extract set null method to the base class for fixed width vectors#5064
liyafan82 wants to merge 1 commit intoapache:masterfrom
liyafan82:fly_0812_null

Conversation

@liyafan82
Copy link
Copy Markdown
Contributor

Currently, each fixed width vector has the setNull method. All these implementations are identical, so we move them to the base class.

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #5064 into master will increase coverage by 1.67%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #5064      +/-   ##
=========================================
+ Coverage   88.02%   89.7%   +1.67%     
=========================================
  Files         919     670     -249     
  Lines      135068   99383   -35685     
  Branches     1418       0    -1418     
=========================================
- Hits       118895   89147   -29748     
+ Misses      16163   10236    -5927     
+ Partials       10       0      -10
Impacted Files Coverage Δ
r/src/recordbatch.cpp
r/R/Table.R
js/src/util/fn.ts
r/src/symbols.cpp
rust/datafusion/src/execution/projection.rs
rust/datafusion/src/execution/filter.rs
rust/arrow/src/csv/writer.rs
rust/datafusion/src/bin/main.rs
js/src/builder/index.ts
r/src/arrow_types.h
... and 240 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5daba72...d654a3d. Read the comment docs.

Copy link
Copy Markdown
Member

@BryanCutler BryanCutler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@BryanCutler
Copy link
Copy Markdown
Member

merged to master, thanks @liyafan82 !

pribor pushed a commit to GlobalWebIndex/arrow that referenced this pull request Oct 24, 2025
…d width vectors

Currently, each fixed width vector has the setNull method. All these implementations are identical, so we move them to the base class.

Closes apache#5064 from liyafan82/fly_0812_null and squashes the following commits:

d654a3d <liyafan82>  Extract set null method to the base class for fixed width vectors

Authored-by: liyafan82 <fan_li_ya@foxmail.com>
Signed-off-by: Bryan Cutler <cutlerb@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants